Skip to content

gh-158837: Clarify csv.reader handling of iterables with stripped newlines. - #158909

Open
utam-1 wants to merge 2 commits into
python:mainfrom
utam-1:gh-158837-csv-newline-docs
Open

utam-1 wants to merge 2 commits into
python:mainfrom
utam-1:gh-158837-csv-newline-docs

Conversation

@utam-1

@utam-1 utam-1 commented Oct 6, 2026 •

Copy link
Copy Markdown

Description

Adds a warning and an example explaining that strings must retain their trailing newlines for quoted multiline values to be preserved.

Related issue

csv.reader do not preserve line return in quoted values

…ed newlines

Adds a warning and an example explaining that strings must retain their trailing newlines for quoted multiline values to be preserved.
@python-cla-bot

python-cla-bot Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.

CLA signed

@read-the-docs-community

read-the-docs-community Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #34969481 | 📁 Comparing 1d0782f against main (82c62ab)

  🔍 Preview build  

1 file changed
± library/csv.html

@utam-1

utam-1 commented Oct 9, 2026

Copy link
Copy Markdown
Author

Hello, could someone please review this PR?

The issue that it's associated with was actually pertaining to documentation. As csv.reader works as expected. I have just clarified the behaviour more prominently along with an example rather than a Footnote.
@picnixz

@picnixz

picnixz commented Oct 9, 2026

Copy link
Copy Markdown
Member

Please avoid pinging maintainers after 3 days only. We have limited time especially due to the sheer number of PRs that are now written by AI. As such, it is not unxommon that noncritical ones take time to be merged (from weeks to months).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review docs Documentation in the Doc dir skip news

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants